Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #40 feat: common pop up's for all assets #75

Merged

Conversation

simran142002
Copy link

@simran142002 simran142002 commented Jul 10, 2023

Functionality:

A common component to handle common ui of video/audio/image pop-up's

Solution:

Introduced a new component named assets browser. Introduced common variables and methods that can be triggered commonly from the html

Risk level:

  • high
  • medium
  • low

Issue:

#40

How to test:

nodemon server.js
npm run start

@simran142002 simran142002 changed the base branch from main to C4GT_Issue_40 July 14, 2023 06:29
@simran142002 simran142002 changed the title feat(codeOptimization): common pop up's for all assets Issue #40 feat: common pop up's for all assets Jul 24, 2023
@vaibhavbhuva vaibhavbhuva merged commit af9cc8c into Sunbird-inQuiry:C4GT_Issue_40 Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants